src/components/therapy-compass/styles.tsx (1)

22-27: 🧹 Nitpick | 🔵 Trivial

Reduced-motion change — confirm UI verification gate was run.

This is a reduced-motion CSS change; per repo guidelines this class of change should be run through npm run ensure and npm run verify:ui (Chromium UI gate) before merge. The PR notes full UI verification wasn't completed locally due to missing optional deps — please confirm CI covers this specific path.

As per coding guidelines: "For UI, frontend, browser, routing, styling, reduced-motion, or forced-colors changes, run npm run ensure before browser work and use npm run verify:ui as the Chromium UI gate."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/components/therapy-compass/styles.tsx` around lines 22 - 27, Confirm that
the reduced-motion styling in the .tc-root .tc-spin and `@keyframes` tc-spin rules
is covered by CI or run npm run ensure followed by npm run verify:ui, and report
the verification status before merge.

Source: Coding guidelines

src/components/therapy-compass/ui.tsx (1)

107-123: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick win

LoadingState isn't announced to assistive tech.

The container has no role="status"/aria-live region, so screen-reader users get no cue when a screen begins/finishes loading (used across search, review queue, detail, etc.).

♿ Proposed fix

 export function LoadingState({ label = "Loading therapy library…" }: { label?: string }) {
   return (
     <div
+      role="status"
+      aria-live="polite"
       style={s(
         `display:flex;flex-direction:column;align-items:center;justify-content:center;gap:12px;min-height:280px;color:var(--text-soft);`,
       )}
     >

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/components/therapy-compass/ui.tsx` around lines 107 - 123, Update the
LoadingState component to make its loading message available to assistive
technology by adding an appropriate status role and live-region behavior to its
outer container. Keep the existing visual structure and label rendering
unchanged.

src/components/therapy-compass/icons.tsx (1)

57-63: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

CompassIcon and TargetIcon are pixel-identical.

Both render the exact same three-circle SVG. Consider aliasing one to the other to avoid drift if either is tweaked later.

♻️ Optional dedupe

-export const TargetIcon = makeIcon(
-  <>
-    <circle cx="12" cy="12" r="9" />
-    <circle cx="12" cy="12" r="5.2" />
-    <circle cx="12" cy="12" r="1.6" fill="currentColor" stroke="none" />
-  </>,
-);
+export const TargetIcon = CompassIcon;

Also applies to: 84-90

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/components/therapy-compass/icons.tsx` around lines 57 - 63, Deduplicate
the identical CompassIcon and TargetIcon definitions by aliasing one exported
symbol to the other. Update CompassIcon or TargetIcon while preserving both
public exports and their existing rendering behavior, so future changes cannot
cause them to drift.

src/components/therapy-compass/bindings.tsx (1)

1-1: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

"Needs review" filter is reimplemented in three places. t.reviewStatus !== "reviewed" is duplicated independently across the nav badge count, the review-queue memo, and the review-queue header count; centralizing it on TcBindings removes the drift risk.

• src/components/therapy-compass/bindings.tsx#L200-269: add a memoized reviewCount/unreviewedTherapies derived from therapies and expose it on the returned value (and TcBindings type).
• src/components/therapy-compass/nav.tsx#L18-18: replace the local reviewCount computation with b.reviewCount (or equivalent) from bindings.
• src/components/therapy-compass/screens/other-screen.tsx#L15-22: base the queue memo on the shared unreviewedTherapies list instead of re-filtering b.therapies.
• src/components/therapy-compass/screens/other-screen.tsx#L76-76: use the shared reviewCount binding instead of recomputing .filter(...).length inline on every render.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/components/therapy-compass/bindings.tsx` at line 1, Centralize the
unreviewed-therapy derivation in the TcBindings value: add memoized
unreviewedTherapies and reviewCount fields based on therapies, and include them
in the TcBindings type and returned value. Update nav.tsx to use b.reviewCount,
and update other-screen.tsx to derive queue and header counts from
b.unreviewedTherapies and b.reviewCount instead of independently filtering
b.therapies.

❌ Autofix failed (check again to retry)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes